♻️ 重构代码，升级为manifest v3#374
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR refactors the project in preparation for Manifest V3, removing legacy configurations and files while upgrading several build and CI workflows.
- Removed legacy files (Babel config, ESLint config, English README) no longer needed under Manifest V3
- Updated README, CONTRIBUTING, and GitHub Actions workflows to use pnpm, Node 22, and newer action versions
- Revised release workflow to streamline asset uploads using a consolidated release action
Reviewed Changes
Copilot reviewed 257 out of 268 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| build/assets/_locales/i18n.js | Removed legacy i18n initialization code |
| babel.config.js | Removed outdated Babel configuration |
| README_EN.md | Removed redundant English documentation |
| README.md | Updated with new setup instructions and project information |
| CONTRIBUTING.md | Updated dependency installation instructions and proxy configuration |
| .github/workflows/test.yaml | Upgraded action versions, node version, and switched to pnpm |
| .github/workflows/packageRelease.yml | Consolidated release steps using a new release action |
| .github/workflows/build.yaml | Updated build workflow with newer actions and pnpm usage |
| .eslintrc.cjs | Removed legacy ESLint configuration |
Files not reviewed (11)
- .eslintignore: Language not supported
- .prettierignore: Language not supported
- .prettierrc: Language not supported
- build/assets/_locales/zh_CN/messages.json: Language not supported
- build/assets/template/background.html: Language not supported
- build/assets/template/confirm.html: Language not supported
- build/assets/template/import.html: Language not supported
- build/assets/template/install.html: Language not supported
- build/assets/template/options.html: Language not supported
- build/assets/template/popup.html: Language not supported
- build/assets/template/sandbox.html: Language not supported
Comments suppressed due to low confidence (2)
.eslintrc.cjs:1
- The ESLint configuration has been entirely removed. Please ensure that linting rules are defined elsewhere to maintain code quality.
module.exports = {
.github/workflows/packageRelease.yml:33
- Switching to ncipollo/release-action@v1 may change the asset upload behavior compared to the previous separate upload steps. Confirm that all expected release assets are correctly uploaded.
- uses: ncipollo/release-action@v1
* 实现 @run-in / 分离隐身模式 @run-in api 显式声明脚本注入标签类型 - `normal-tabs` 注入正常标签 - `incognito-tabs` 注入隐身标签 缺省时、同时声明时，既注入正常标签又隐身标签 具体用法见新增示例 新增 `CacheStorage.prototype.clear` 清除所有缓存 同步更新`GM_info` `manifest.json`显示声明`"incognito": "split"`分离浏览器隐身模式时扩展进程及上下文 `package.json`新增`dev-noMap`方法 - 无`SourceMap`的`development`模式 （已知带`SourceMap`的`dev`模式会造成扩展的`service_work.js`无法完成注册，可能是浏览器限制或bug） * 优化 仅隐身模式第一次初始化清理session * fix * fix allFrames * 优化 run-in 实现 * 恢复逻辑 getUserScriptRegister 实际是get+set * 恢复 Promise.all
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
IndexedDB为chrome.storage.local